Skip to content

Conversation

@cnotv
Copy link
Member

@cnotv cnotv commented Oct 10, 2025

Summary

Fixes #15510
Fixes #15579

Occurred changes and/or fixed issues

Restricted navigation to admin only and feature flag.

Technical notes summary

  • Added FF rancher-scc-registration-extension
  • Added extensions unit tests (first project case)
    • Role must be identified by Rancher as admin
    • Feature flag must be passed by Rancher

Areas or cases that should be tested

Areas which could experience regressions

NOTE: If you stop and restart the feature flag in Docker it will make Rancher crash. Use a new instance to test this out if you have a lot of data.

Navigation to the registration page using the UI or both admin and standard user.

Screenshot/Video

Comparison admin to standard

Screenshot 2025-10-14 at 16 32 20

Navigation and notifications are missing if no feature as admin

Screenshot 2025-10-14 at 17 35 17

Checklist

  • The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
  • The PR has a Milestone
  • The PR template has been filled out
  • The PR has been self reviewed
  • The PR has a reviewer assigned
  • The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
  • The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes
  • The PR has been reviewed in terms of Accessibility

@cnotv cnotv force-pushed the bugfix/15510-scc-rbac branch from fde60b2 to 18b4c27 Compare October 15, 2025 12:56
@cnotv cnotv requested review from aalves08 and torchiaf October 15, 2025 17:41
@cnotv cnotv marked this pull request as ready for review October 15, 2025 17:45
Copy link
Member

@aalves08 aalves08 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gave it a test and works as advertised, so LGTM!

@aalves08
Copy link
Member

@cnotv created #15645 as a spin-off of your PR 🙏

@cnotv cnotv merged commit 9b1832b into rancher:master Oct 16, 2025
62 of 64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rancher UI is showing system is not with SCC registered when it is Prevent access of non-admin users to SCC Registration page and notification

2 participants